Skip to content

fix(stapp): remember selected llm#348

Closed
szdtzpj wants to merge 1 commit into
lsdefine:mainfrom
szdtzpj:codex/persist-stapp-llm-selection
Closed

fix(stapp): remember selected llm#348
szdtzpj wants to merge 1 commit into
lsdefine:mainfrom
szdtzpj:codex/persist-stapp-llm-selection

Conversation

@szdtzpj
Copy link
Copy Markdown

@szdtzpj szdtzpj commented May 12, 2026

Summary

  • expand the Streamlit sidebar on startup
  • persist the selected LLM index in temp/stapp_state.json
  • restore the saved LLM before the frontend worker starts
  • tolerate UTF-8 BOM if the state file is edited by Windows tools

This covers the sidebar-default and model-selection-memory parts of #200 without adding dependencies.

Testing

  • conda env: ga-stapp-verify, Python 3.12, Streamlit 1.57.0
  • python -m py_compile frontends\stapp.py agentmain.py llmcore.py
  • python -m unittest discover -s tests -v
  • launched Streamlit at http://127.0.0.1:18652
  • Playwright + Microsoft Edge: verified sidebar aria-expanded=true, width=300
  • Playwright: preloaded temp/stapp_state.json with llm_no=1 and verified UI restored verify-secondary
  • Playwright: selected verify-primary in the sidebar and verified temp/stapp_state.json updated to llm_no=0

@szdtzpj szdtzpj force-pushed the codex/persist-stapp-llm-selection branch from aef3091 to cbf3c01 Compare May 12, 2026 10:21
@szdtzpj
Copy link
Copy Markdown
Author

szdtzpj commented May 13, 2026

Superseded by #355.

@szdtzpj szdtzpj closed this May 13, 2026
@szdtzpj szdtzpj deleted the codex/persist-stapp-llm-selection branch May 13, 2026 02:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant